Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename instances of appdata to metainfo #442

Merged
merged 4 commits into from
Feb 2, 2024
Merged

Conversation

bbhtt
Copy link
Collaborator

@bbhtt bbhtt commented Feb 1, 2024

See flathub-infra/documentation#208

Most of the content of freedesktop-quick-reference is already in the conventions page and it is essentially a duplicate.

@@ -107,7 +107,8 @@ build like this is very fast.

In order for your application to show up in application stores while testing with a local repository, you might have
to run ``flatpak build-update-repo repo``.
For more information how to publish to application stores see `Appdata files <https://docs.flatpak.org/en/latest/freedesktop-quick-reference.html#appdata-files>`_.

For more information how to publish to application stores see `MetaInfo files <https://docs.flatpak.org/en/latest/conventions.html#metainfo-files>`_.
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The anchor being broken here should be an expected CI failure

Delete the former as most of the content is now in conventions page.

Also add more info about Flatpak icon/desktopfile exports
@TingPing TingPing merged commit ced51bd into flatpak:master Feb 2, 2024
1 of 2 checks passed
@bbhtt bbhtt deleted the fix-appdata branch February 2, 2024 16:49
@bbhtt
Copy link
Collaborator Author

bbhtt commented Feb 2, 2024

@TingPing looks like the link needs to be switched back and then once it is deployed needs to be switched to the correct one.

Otherwise master branch is failing and not being deployed I think

@bbhtt
Copy link
Collaborator Author

bbhtt commented Feb 2, 2024

Nevermind, sorry looks like it worked now.

@TingPing
Copy link
Member

TingPing commented Feb 2, 2024

I had to fix webhooks because of a recent readthedocs change. Deployment isn't done by Github so status doesn't matter here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants